Conversation
| String name; | ||
| int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне. Тогда можно удалить геттеры и обращаться к полям напрямую
|
|
||
| while (true) { | ||
| System.out.println("Введите имя гонщика: "); | ||
| inputName = scanner.nextLine(); |
There was a problem hiding this comment.
Лучше тут ещё воспользоваться функцией trim(), чтобы не позволять вводить название машин, состоящие только из пробелов и переносов строк
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| Scanner scanner = new Scanner(System.in); | ||
| ArrayList<Auto> autos = new ArrayList<>(); |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| inputSpeed = scanner.nextInt(); | ||
| scanner.nextLine(); | ||
|
|
||
| if (inputSpeed > 0 && inputSpeed < 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
2 participants
No description provided.